Skip to content

Verify that files generated with SwiftSyntaxBuilder match the checked-in state #1117

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Dec 6, 2022

Currently, we only verify that the gyb-generated files match the checked in source state. As more and more files are being generated using CodeGeneration, we should also verify them in CI.

rdar://103023853

@ahoppen ahoppen force-pushed the ahoppen/verify-codegened-files branch from 09d75c8 to c0567a8 Compare December 6, 2022 15:03
@ahoppen
Copy link
Member Author

ahoppen commented Dec 6, 2022

@swift-ci Please test

…-in state

"Currently, we only verify that the gyb-generated files match the checked in source state. As more and more files are being generated using CodeGeneration, we should also verify them in CI.

rdar://103023853
@ahoppen ahoppen force-pushed the ahoppen/verify-codegened-files branch from fdcb9ee to 1f82dd4 Compare December 7, 2022 21:29
@ahoppen
Copy link
Member Author

ahoppen commented Dec 7, 2022

swiftlang/swift#62445

@swift-ci Please test

@ahoppen ahoppen requested a review from bnbarham December 7, 2022 21:29
@ahoppen ahoppen marked this pull request as ready for review December 7, 2022 21:29
@ahoppen ahoppen changed the title WIP: Verify that files generated with SwiftSyntaxBuilder match the checked-in state Verify that files generated with SwiftSyntaxBuilder match the checked-in state Dec 8, 2022
@ahoppen
Copy link
Member Author

ahoppen commented Dec 8, 2022

swiftlang/swift#62445

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Apr 28, 2023

Closing in favor of #1546

@ahoppen ahoppen closed this Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants